Skip to content

Fix unintended verbose output when running rake -m - #538

Merged
kodiakhq[bot] merged 1 commit into
mainfrom
clean-test-output
Jul 15, 2026
Merged

Fix unintended verbose output when running rake -m#538
kodiakhq[bot] merged 1 commit into
mainfrom
clean-test-output

Conversation

@mattbrictson

@mattbrictson mattbrictson commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Problem

Running rake -m would for some reason switch Minitest into verbose mode, showing the full name of each test, like this:

# Running:

Tomo::PathTest#test_join = 0.00 s = .
Tomo::PathTest#test_dirname = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_does_not_overwrite_previously_generated_secret = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_setup_fails_if_wrong_envrc_already_exists_in_bashrc = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_setup_does_not_modify_bashrc_if_it_is_already_set_up = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_setup_generates_a_random_value_for_var_marked_as_generate_secret = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_executes_chmod_to_reduce_visibility_of_envrc_file_upon_creation = 0.00 s = .
Tomo::Plugin::Env::TasksTest#test_setup_allows_integer_value = 0.00 s = .

Solution

Use Minitest::TestTask instead of Rake::TestTask. This shows the intended output when using rake -m.

# Running:

.............................................

@mattbrictson mattbrictson added 🏠 Housekeeping Non-user facing cleanup and maintenance automerge Automatically merge this PR once all required checks pass labels Jul 15, 2026
@kodiakhq
kodiakhq Bot merged commit b748fb5 into main Jul 15, 2026
9 checks passed
@kodiakhq
kodiakhq Bot deleted the clean-test-output branch July 15, 2026 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

automerge Automatically merge this PR once all required checks pass 🏠 Housekeeping Non-user facing cleanup and maintenance

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mattbrictson